KMS: atomic modeset/pageflip 


Rob Clark 


What is it? 


e Atomic pageflip 
— Updating CRTC fb and/or one or more plane fb's atomically (in a single 
vblank) 


— Also possibly adjusting properties: z-order, alpha blending modes, rotation, 
colorspace-conversion coefficients, etc 


— 'test' flag to allow userspace to check a proposed configuration first 


e Atomic modeset 
— Configuring one or more CRTCs 


— 'test' flag to allow checking if the proposed combination of 
timings/resolutions are supported by the hw 
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Why do we need it? 


e Atomic pageflip 
— Compositors using overlay planes to bypass GPU for compositing surfaces 


e Need to keep bypassed surface state (size, position, fb) in sync w/ GPU 
composition output on CRTC layer 
e Need to know when they'll hit hw limits about overlay plane sizes/scaling/etc 
— Some limits may be with combinations of multiple enabled planes 
— So not easy to express limits statically to userspace 


e Atomic modeset 
— Userspace needs to know valid combinations of settings for multi-display 


— Memory bandwidth limits, etc, may mean that certain resolutions are 
possible with single display but not multiple displays 
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Property-ification.. 


e The proposed solution configures “everything” via properties 
e We need to support taking a list of properties anyways 


e Doing everything via properties means: 
— common code-paths 
— Future extensibility 


e But then how does error checking work? 
— le. valid fo dimensions, position, etc 
— Short version: it is still there, but moves from the ioctl handler fxn 
— Long version: on next slides 
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Splitting mode object mutable state 


e What is in 'struct drm_{crtc,plane,etc}' is combination of: 
— Mode state set from userspace: fb, {src,crtc} {x,y,w,h}, etc 
— Other: possible_crtcs, list head, funcs, etc 


e For 'test' steps, we need to build up proposed state, and rollback 


— Split into 'struct drm_{crtc,plane,etc} state’ simplifies things 
° Just a single pointer to update to commit changes 
e We could probably simplify crtc helpers change rollback this way 
— Split out of state structs also lets us use helpers to: 
e Avoid a lot of property nonsense in each driver for common properties 
e Re-introduce the standard error checking lost from ioctl handler 


int drm_plane_check_state(struct drm_plane *plane, struct drm_plane_state *state); 

void drm_plane_commit_state(struct drm_plane *plane, struct drm_plane_state *state); 

int drm_plane_set_property(struct drm_plane *plane, struct drm_plane_state *state, 
struct drm_property *property, uint64_t value); 


e (And same for CRTC and eventually connector) 
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Splitting mode object mutable state (cont) 


e Also, property values array moved into state structs 
— Automatically keeps userspace visible property values in sync 
— Don't get property values confused by 'test' step or failed config changes 


struct drm_plane_state { 
struct drm_crtc *crtc; 
struct drm_framebuffer *fb; 


/* Signed dest location allows it to be partially off screen */ 
int32_t erte x, crtc_y; 
uint32_t crtc_w, crtc_h; 


/* Source values are 16.16 fixed point */ 
uint32_t src_x, src_y; 
uint32_t src_h, src_w; 


struct drm_object_property_values propvals; 
}; 


e Drivers should wrap state structs w/ their own to add driver specifics: 


struct omap_ plane state { 
struct drm plane state base; 
uint8_t rotation; 
uint8_t zorder; 


}, 
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Atomic funcs 


atomic_begin(dev) - allocate state token 


atomic_check(dev, state) — check proposed state 
— Use drm_*_check_state() for common stuff 


atomic_commit(dev, state) — commit proposed state 
— Do driver specific stuff, then drm_* commit_state() 


atomic_end(dev, state) — cleanup/deallocate 


Example: 
state = dev->driver->atomic_begin(dev); 


if (page_flip->flags & DRM_MODE_PAGE_FLIP_EVENT) 


e = create_vblank_event(dev, file_priv, page_flip->user_data); 


for (i = 0; i < page_flip->count_props; i++) 
drm_mode_set_obj_prop_id(dev, state, 
prop.obj_id, prop.obj_type, 
prop.prop_id, prop.value); 


ret = dev->driver->atomic_check(dev, state); 


if (!(page_flip->flags & DRM_MODE_TEST_ONLY) ) 
ret = dev->driver->atomic_commit(dev, state, e); 


dev->driver->atomic_end(dev, state); 
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Other misc changes 


e Object property type 
— DRM_MODE_PROP_OBJECT 
— To set crtc, fb, etc as a property 


e Dynamic property flag 
- DRM_MODE_PROP_DYNAMIC 


— Hint to userspace about properties which can be safely changed without 
‘test’ step 


e Signed property ranges 
— DRM_MODE_PROP_SIGNED 
— Uses signed integer comparison to check for valid property values 
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TODO 


e Don't remove plane->update_plane(), crtc->page_flip() yet 
— These are no longer needed for 'property-ified' drivers 
— But probably better to have a transition period, than port all drivers at once 


e Still tweaking ioctl struct 
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Questions? 
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